app/src/main/java/com/texthip/thip/ui/common/modal/ScrollbarUtil.kt (2)

61-61: 고정된 스크롤바 높이가 컨텐츠 비율을 반영하지 않습니다.

scrollbarHeight = size.height / 8f로 고정되어 있어, 실제 컨텐츠 양과 뷰포트 크기의 비율을 반영하지 못합니다. 일반적인 스크롤바 구현에서는 썸 높이가 (뷰포트 높이 / 전체 컨텐츠 높이) * 뷰포트 높이로 계산됩니다.

다음과 같이 개선할 수 있습니다:

-        val scrollbarHeight = size.height / 8f
+        val thumbHeightRatio = (viewportHeight / estimatedTotalHeight.toFloat()).coerceIn(0.1f, 1f)
+        val scrollbarHeight = size.height * thumbHeightRatio

---

84-97: 두 오버로드 간 스크롤바 그리기 로직이 중복됩니다.

ScrollState와 LazyListState 오버로드 모두 동일한 drawRoundRect 호출을 사용하여 트랙과 썸을 그립니다. 이 로직을 별도의 private 함수로 추출하면 코드 중복을 제거하고 유지보수성이 향상됩니다.

다음과 같이 리팩토링할 수 있습니다:

private fun DrawScope.drawScrollbar(
    trackThickness: Dp,
    thumbThickness: Dp,
    trackColor: Color,
    thumbColor: Color,
    scrollbarOffsetY: Float,
    scrollbarHeight: Float
) {
    // 전체 고정 바
    drawRoundRect(
        color = trackColor,
        topLeft = Offset(x = size.width - trackThickness.toPx(), y = 0f),
        size = Size(trackThickness.toPx(), size.height),
        cornerRadius = CornerRadius(trackThickness.toPx() / 2)
    )
    // 핸들 바
    drawRoundRect(
        color = thumbColor,
        topLeft = Offset(x = size.width - thumbThickness.toPx(), y = scrollbarOffsetY),
        size = Size(thumbThickness.toPx(), scrollbarHeight),
        cornerRadius = CornerRadius(thumbThickness.toPx() / 2)
    )
}

그런 다음 두 오버로드에서 이 함수를 호출하면 됩니다.

app/src/main/java/com/texthip/thip/ui/group/makeroom/component/GroupDatePicker.kt (1)

114-114: 방어적 코딩이 적용되었으나 실질적으로는 불필요합니다.

days.maxOrNull() ?: 1을 사용하여 null 안전성을 추가했습니다. days 리스트는 (1..lengthOfMonth()).toList()로 생성되므로 항상 최소 1개 이상의 요소를 포함하여 maxOrNull()이 null을 반환할 수 없습니다.

현재 코드도 문제없지만, 더 명확하게 하려면 days.max()를 그대로 사용하거나, 주석으로 방어적 코딩 의도를 명시하는 것을 고려할 수 있습니다.

app/src/main/java/com/texthip/thip/ui/group/makeroom/viewmodel/GroupMakeRoomUiState.kt (1)

45-47: 비밀번호 형식 검증 보완 제안(숫자 4자리 제한)

현재 길이(4)만 확인하여 영문/기호도 통과합니다. 숫자 4자리만 허용하려면 아래처럼 보완하는 것을 권장합니다.

-    val isPasswordValid: Boolean
-        get() = !isPrivate || password.length == 4
+    val isPasswordValid: Boolean
+        get() = !isPrivate || (password.length == 4 && password.all { it.isDigit() })

app/src/main/java/com/texthip/thip/ui/common/modal/DialogPopup.kt (1)

37-43: 고정 폭 320.dp → 반응형 제약으로 변경 권장

작은/큰 화면 대응을 위해 fillMaxWidth + widthIn(max)를 권장합니다. 현재 고정 폭은 태블릿/분할 화면에서 부자연스러울 수 있습니다.

적용 diff:

@@
-            .width(320.dp)
+            .fillMaxWidth()
+            .widthIn(max = 320.dp)
             .background(
                 color = colors.DarkGrey,
                 shape = RoundedCornerShape(12.dp)
             )
             .padding(20.dp)

필요 시 import:

+import androidx.compose.foundation.layout.widthIn

app/src/main/java/com/texthip/thip/ui/group/makeroom/screen/GroupMakeRoomScreen.kt (3)

166-171: GenreChipRow에 width(12.dp) 적용은 오타/의도 오해 가능성 높음

Row 너비를 12dp로 제한하면 칩이 잘리거나 배치가 비정상일 수 있습니다. 전체 폭 사용을 권장합니다.

제안 diff:

-                GenreChipRow(
-                    modifier = Modifier.width(12.dp),
+                GenreChipRow(
+                    modifier = Modifier.fillMaxWidth(),
                     genres = uiState.genres.toDisplayStrings(),
                     selectedIndex = uiState.selectedGenreIndex,
                     onSelect = onSelectGenre,
                     horizontalArrangement = Arrangement.Center
                 )

---

126-130: blur 조건이 많으면 저사양 기기에서 성능 저하 가능

스크림(반투명 배경)만으로도 강조가 충분한 경우가 많습니다. blur 강도를 낮추거나 조건을 축소하는 방안을 고려해 주세요.

---

54-55: DATE_FORMATTER 로케일/중복 관리

고정 패턴 대신 Locale에 맞춘 형식(또는 공용 유틸의 포맷터)을 재사용하면 일관성과 i18n 대응이 좋아집니다. utils에 이미 포맷터가 있다면 그쪽으로 통합 권장.

app/src/main/java/com/texthip/thip/ui/group/makeroom/component/GroupBookSearchBottomSheet.kt (3)

31-31: 테마 색상 접근 방식 일관화 + 인디케이터 가독성 유지

colors alias와 ThipTheme.colors를 혼용하면 가독성이 떨어집니다. 한 가지 방식으로 통일하는 것을 권장합니다. 아래처럼 alias import를 제거하고 ThipTheme.colors를 사용해 주세요.

- import com.texthip.thip.ui.theme.ThipTheme.colors

-                            CircularProgressIndicator(
-                                color = colors.White
-                            )
+                            CircularProgressIndicator(
+                                color = ThipTheme.colors.White
+                            )

추가로 로딩 시 수직 중앙정렬을 의도하셨다면 Box에 동일한 높이 정책을 부여해야 합니다(현재는 가로 중앙 정렬만 적용). 예시:

Box(
    modifier = Modifier
        .fillMaxWidth()
        .weight(1f) // <- 추가 (ColumnScope)
        .align(Alignment.CenterHorizontally),
    contentAlignment = Alignment.Center
) { CircularProgressIndicator(color = ThipTheme.colors.White) }

Also applies to: 125-127

---

132-136: 로딩/목록 레이아웃의 높이/패딩 일관성 확보

목록 분기에는 weight(1f)와 좌우 padding을 주셨지만, 로딩 분기(Box)에는 없어 상·하 위치가 달라 보일 수 있습니다. 상단 입력 영역 고정, 하단 콘텐츠 영역(로딩/목록 동일)로 레이아웃을 통일하면 시각적 점프가 줄어듭니다.

간단히는 로딩 Box에도 동일한 weight/패딩을 부여하거나, 아래처럼 공통 컨테이너로 감싸세요:

val contentModifier = Modifier
    .weight(1f)
    .padding(horizontal = 20.dp)

if (isLoading) {
    Box(Modifier.fillMaxWidth().then(contentModifier), contentAlignment = Alignment.Center) { ... }
} else {
    Column(Modifier.fillMaxWidth().then(contentModifier)) { ... }
}

---

53-54: API 추가 사항 정리: showGroupBooksTab 기본값 문서화 및 사용처 검증

기획상 모임 생성 플로우에서는 그룹 책 탭을 숨겨야 하므로, 해당 화면 호출부에서 showGroupBooksTab=false가 실제로 적용되는지 확인해 주세요. 또한 파라미터 의도를 KDoc으로 명시하면 재사용 시 혼동을 줄일 수 있습니다. 프리뷰에도 false 케이스를 하나 추가하는 것을 권장합니다.

프리뷰 예시:

@Preview(showBackground = true)
@Composable
fun PreviewBookSearchBottomSheet_SavedOnly() {
    ThipTheme {
        GroupBookSearchBottomSheet(
            onDismiss = {},
            onBookSelect = {},
            onRequestBook = {},
            savedBooks = dummySavedBooks,
            groupBooks = emptyList(),
            showGroupBooksTab = false
        )
    }
}

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

app/src/main/java/com/texthip/thip/ui/group/makeroom/screen/GroupMakeRoomScreen.kt (3)

292-316: Box 오버레이 → Compose Dialog 전환, 접근성/포커스/뒤로가기 보장

Dialog로 교체되어 시스템 레벨 동작이 확보되었습니다. onDismissRequest 처리도 적절합니다. 이전 리뷰 지적을 잘 반영하셨습니다.

---

299-304: 문자열 포맷 인자 중복(meetingStartDate 2회) — 모집 종료일을 전달해야 합니다.

strings.xml의 group_create_confirm_message는 4개 인자를 기대하며, 2번째는 “모집 종료일”이어야 합니다. 현재 2번째·3번째에 meetingStartDate가 중복 전달됩니다. UiState에 recruitmentEndDate(또는 동등 필드)를 추가·관리한 뒤 아래처럼 교체하세요.

                     description = stringResource(
                         R.string.group_create_confirm_message,
                         java.time.LocalDate.now().format(DATE_FORMATTER),
-                        uiState.meetingStartDate.format(DATE_FORMATTER),
+                        uiState.recruitmentEndDate.format(DATE_FORMATTER),
                         uiState.meetingStartDate.format(DATE_FORMATTER),
                         uiState.meetingEndDate.format(DATE_FORMATTER)
                     ),

검증 스크립트(플레이스홀더 순서 확인):

#!/bin/bash
# strings.xml에서 group_create_confirm_message 확인
fd -a strings.xml | while read f; do
  echo "== $f"
  rg -nC2 'name="group_create_confirm_message"' "$f"
done

---

318-327: 로딩 오버레이가 터치를 차단하지 않습니다 → 중복 입력/오작동 위험

반투명 Box는 포인터를 소비하지 않습니다. 로딩 중에는 입력을 반드시 인터셉트하세요.

@@
-import androidx.compose.ui.graphics.Color
+import androidx.compose.ui.graphics.Color
+import androidx.compose.foundation.clickable
+import androidx.compose.foundation.interaction.MutableInteractionSource
+import androidx.compose.runtime.remember
@@
-        if (uiState.isLoading) {
+        if (uiState.isLoading) {
             Box(
                 modifier = Modifier
                     .fillMaxSize()
-                    .background(Color.Black.copy(alpha = 0.5f)),
+                    .background(Color.Black.copy(alpha = 0.5f))
+                    .clickable(
+                        indication = null,
+                        interactionSource = remember { MutableInteractionSource() }
+                    ) { /* consume clicks while loading */ },
                 contentAlignment = Alignment.Center
             ) {
                 CircularProgressIndicator(color = colors.NeonGreen)
             }
         }

app/src/main/java/com/texthip/thip/ui/group/makeroom/component/GroupDatePicker.kt (3)

62-75: 예외 기반 폴백 대신 계산 기반으로 단순화하세요.

정상 흐름에서 try/catch는 불필요한 오버헤드입니다. 해당 월의 길이로 안전한 day를 계산해 바로 생성하면 가독성과 성능이 개선됩니다. 월 변경 블록(라인 95-101)에도 동일 적용을 권장합니다.

+import kotlin.math.min
@@
-                    val fallbackDate = try {
-                        LocalDate.of(newYear, month, day)
-                    } catch (e: Exception) {
-                        val lastDay = LocalDate.of(newYear, month, 1).lengthOfMonth()
-                        LocalDate.of(newYear, month, lastDay)
-                    }
+                    val lastDay = LocalDate.of(newYear, month, 1).lengthOfMonth()
+                    val fallbackDate = LocalDate.of(newYear, month, min(day, lastDay))

월 변경 시:

-                    val fallbackDate = try {
-                        LocalDate.of(year, newMonth, day)
-                    } catch (e: Exception) {
-                        val lastDay = LocalDate.of(year, newMonth, 1).lengthOfMonth()
-                        LocalDate.of(year, newMonth, lastDay)
-                    }
+                    val lastDay = LocalDate.of(year, newMonth, 1).lengthOfMonth()
+                    val fallbackDate = LocalDate.of(year, newMonth, min(day, lastDay))

---

68-74: 클램프 로직 중복 → 헬퍼로 추출해 DRY 유지.

min/maxDate 보정 when 블록이 연-월-일에 반복됩니다. private 헬퍼(예: clamp(date, min, max))로 추출하거나 기존 공용 유틸이 있다면 재사용해 단일 소스로 관리하세요. 유지보수 시 실수 가능성이 줄어듭니다.

+private fun clamp(date: LocalDate, minDate: LocalDate, maxDate: LocalDate): LocalDate =
+    when {
+        date.isBefore(minDate) -> minDate
+        date.isAfter(maxDate) -> maxDate
+        else -> date
+    }
@@
-                    val validatedDate = when {
-                        fallbackDate.isBefore(minDate) -> minDate
-                        fallbackDate.isAfter(maxDate) -> maxDate
-                        else -> fallbackDate
-                    }
+                    val validatedDate = clamp(fallbackDate, minDate, maxDate)
@@
-                    val validatedDate = when {
-                        fallbackDate.isBefore(minDate) -> minDate
-                        fallbackDate.isAfter(maxDate) -> maxDate
-                        else -> fallbackDate
-                    }
+                    val validatedDate = clamp(fallbackDate, minDate, maxDate)
@@
-                    val validatedDate = when {
-                        newDate.isBefore(minDate) -> minDate
-                        newDate.isAfter(maxDate) -> maxDate
-                        else -> newDate
-                    }
+                    val validatedDate = clamp(newDate, minDate, maxDate)

Also applies to: 102-106, 130-134

---

43-49: 월/일 목록을 min/maxDate에 맞춰 동적 제한 권장(UX).

현재는 불가능한 월/일도 노출됐다가 선택 시점에 클램프됩니다. 선택 목록 자체를 범위에 맞추면 혼란이 줄어듭니다.

-    val months = remember { (1..12).toList() }
-    val days = remember(year, month) {
-        (1..LocalDate.of(year, month, 1).lengthOfMonth()).toList()
-    }
+    val months = remember(year, minDate, maxDate) {
+        when (year) {
+            minDate.year -> (minDate.monthValue..12).toList()
+            maxDate.year -> (1..maxDate.monthValue).toList()
+            else -> (1..12).toList()
+        }
+    }
+    val days = remember(year, month, minDate, maxDate) {
+        val last = LocalDate.of(year, month, 1).lengthOfMonth()
+        val start = if (year == minDate.year && month == minDate.monthValue) minDate.dayOfMonth else 1
+        val end = if (year == maxDate.year && month == maxDate.monthValue) minDate.withYear(year).withMonth(month).let { 
+            maxDate.dayOfMonth 
+        } else last
+        (start..end).toList()
+    }

또한 day 선택 항목을 다음처럼 보정하면 일관성이 높습니다:

-                selectedItem = day.coerceAtMost(days.maxOrNull() ?: 1),
+                selectedItem = day.coerceIn(days.firstOrNull() ?: 1, days.lastOrNull() ?: 1),

app/src/main/java/com/texthip/thip/ui/group/makeroom/screen/GroupMakeRoomScreen.kt (1)

166-172: 가로 중앙 정렬 의도라면 고정 width(12.dp) 제거가 필요합니다.

현재 Modifier.width(12.dp)로 영역이 12dp로 제한되어 Center가 무의미합니다. 리스트 폭을 확보하세요.

-                GenreChipRow(
-                    modifier = Modifier.width(12.dp),
+                GenreChipRow(
+                    modifier = Modifier.fillMaxWidth(),
                     genres = uiState.genres.toDisplayStrings(),
                     selectedIndex = uiState.selectedGenreIndex,
                     onSelect = onSelectGenre,
                     horizontalArrangement = Arrangement.Center
                 )

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.